]> dgit.raspbian.org Git - gnome-shell.git/commit
authPrompt: Disregard smartcard status changes if VERIFICATION_IN_PROGRESS
authorRay Strode <rstrode@redhat.com>
Mon, 15 May 2023 14:48:15 +0000 (10:48 -0400)
committerFlorian Müllner <fmuellner@gnome.org>
Sat, 17 Feb 2024 14:04:04 +0000 (14:04 +0000)
commit3dd5dcd9bba2a828ffe9468e55132e3191c0011d
tree9f17c68a1dbd7d4b18df587e0064a16022adb9df
parentaebef54a1d3cb7505e753e280fb90f88138d90df
authPrompt: Disregard smartcard status changes if VERIFICATION_IN_PROGRESS

commit c8bb45b41c3a13ef161103f649aa18938e028a70 introduced a new
verification state, VERIFICATION_IN_PROGRESS, to detect when the user
has already interacted with the authentication prompt, so the
prompt can rate limit the number of times the user can cancel
authentication attempts with the escape key (without also rate limiting
the number of times they can hit escape to go back to the clock without
interacting with the prompt).

That means there are now two states that represent the user actively
undergoing verification: VERIFYING and VERIFICATION_IN_PROGRESS.

It's inappropriate to reset the smartcard service if the user is
actively conversing with it. We try to check for that by looking at the
original verification state, VERIFYING, but we unfortunately, neglect
to account for the new VERIFICATION_IN_PROGRESS state.

The result is that if a user types their smartcard pin at the clock,
and then inserts their smartcard, the pin will get cleared instead of
used, and they have to retype it again.

This commit fixes the oversight, and allows users to again pre-type
their smartcard pin at the clock before inserting their smartcard.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/2765>
js/gdm/authPrompt.js